Skip to content
This repository has been archived by the owner on Mar 29, 2020. It is now read-only.

pom profile #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

pom profile #27

wants to merge 1 commit into from

Conversation

f4tq
Copy link

@f4tq f4tq commented Jun 2, 2011

Hi,
Thanks for the trqbox-demo. I'm porting a Seam 2 project to jruby via torquebox so I immediately started trying to extend your demo. Once I added a maven dependency for softwaremill-cdi, I hit this library compatability issue. Since this is particular to a jboss container, I'm not sure what course you would prefer to handle this but maybe via a maven profile? I imagine a lot of people use jboss 6.0 or torquebox 1.1.

My Commit Message:

Reference TorqueBox 1.1/jboss 6.0's Weld 1.1.0.Final version. More importantly, regress softwaremill-common back to guava r06 because the jbossas 6 classloader throws 'Unsatisfied link' exception loading libraries that otherwise work just fine with trqbox-demo tests. softwaremill-cdi was explicitely using guava r07. This will be frustrating to anyone trying to follow and extend the demo to intergrate other CDIs into torquebox. All tests worked on the regression/installation of 47-SNAPSHOT.

Thanks,
Chris

…portantly, regress softwaremill-common back to guava r06 because the jbossas 6 classloader throws 'Unsatisfied link' exception loading libraries that otherwise work just fine with trqbox-demo tests. softwaremill-cdi was explicitely using guava r07. This will be frustrating to anyone trying to follow and extend the demo to intergrate other CDIs into torquebox. All tests worked on the regression/installation of 47-SNAPSHOT
@adamw
Copy link
Member

adamw commented Jun 8, 2011

Ah, right, we just work around this issue by marking guava as provided in our pom and copying it to jboss/server/default/lib. But in the demo's pom.xml that's already done, I guess you must have hit the issue when trying to re-create the demo?
Anyway, I think the best thing to do here would be to add a note to the README, how do you think!

Thanks for trying out the demo, btw.! :)

Adam

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants